docs(hoppscotch): sync template standards updates by mberlofa · Pull Request #353 · helmforgedev/site

mberlofa · 2026-07-03T23:06:12Z

Summary

sync Hoppscotch docs for the template-standards chart update
document networkPolicy.extraEgress and extraManifests
correct the documented image tag to 2026.5.0
add Hoppscotch to the playground site-sync registry and expose network policy controls

Validation

npm run lint
npm run format:check
npm run build
make site-sync-check CHART=hoppscotch
make release-check REPO=site
make attribution-check REPO=site

Chart PR: helmforgedev/charts#675

Summary by CodeRabbit

New Features
- Added the Hoppscotch chart to the playground list when eligible.
- Introduced an optional Network Policy “Network Policy” section (shown when enabled) with extra egress controls for destination CIDRs and ports.
Documentation
- Updated Hoppscotch Helm chart docs to use image tag 2026.5.0.
- Documented the new Helm parameters, including networkPolicy.extraEgress and extraManifests.

coderabbitai · 2026-07-03T23:06:32Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 05c118c1-e116-4de9-9f22-91c5c6120261

📥 Commits

Reviewing files that changed from the base of the PR and between 99dcbdb and e8ad4d3.

📒 Files selected for processing (3)

src/data/playground-configs.ts
src/pages/docs/charts/hoppscotch.mdx
src/pages/playground.astro

🚧 Files skipped from review as they are similar to previous changes (2)

src/pages/playground.astro
src/data/playground-configs.ts

📝 Walkthrough

Walkthrough

This PR adds a Network Policy section to the Hoppscotch playground config, registers Hoppscotch in the playground mapping, and updates the Hoppscotch chart docs with a newer image tag plus additional documented parameters.

Changes

Hoppscotch Network Policy playground and docs

Layer / File(s)	Summary
Network Policy playground config and wiring `src/data/playground-configs.ts`, `src/pages/playground.astro`	A collapsible Network Policy section gated by `networkPolicy.enabled` adds extra egress CIDR and port fields, and the `hoppscotch` config is added to the playground mapping.
Chart documentation updates `src/pages/docs/charts/hoppscotch.mdx`	The intro and parameters table now reference image tag `2026.5.0`, and the table adds `networkPolicy.extraEgress` and `extraManifests` defaults.

Estimated code review effort: 2 (Simple) | ~10 minutes

Possibly related PRs

helmforgedev/site#330: Also extends src/pages/playground.astro’s chart registry with another playground chart slug.

🚥 Pre-merge checks | ✅ 5

✅ Passed checks (5 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title is concise and accurately reflects the Hoppscotch docs sync and playground registry update.
Description check	✅ Passed	The description covers the summary and validation, but it omits the Related Issue field and doesn't use the template's checkbox format.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

✨ Finishing Touches

📝 Generate docstrings

Create stacked PR
Commit on current branch

🧪 Generate unit tests (beta)

Create PR with unit tests
Commit unit tests in branch docs/hoppscotch-template-standards-sync

_{Comment @coderabbitai help to get the list of available commands.}

mberlofa · 2026-07-05T18:17:21Z

Synced this site PR with the Hoppscotch chart change in helmforgedev/charts#675.

What changed:

Updated the Hoppscotch values reference to document that extraManifests are rendered with Helm tpl.

Validation:

make site-sync-check CHART=hoppscotch reports the required artifacts as OK.
npm run format:check passed.
npm run lint passed.
npm run build passed.
make release-check REPO=site passed.
make attribution-check REPO=site passed.

## Summary - Keep the Helm test hook under `templates/tests/connection-test.yaml` so `helm test` renders and executes it, with unittest coverage for the hook pod. - Add `networkPolicy.extraEgress` while preserving legacy `networkPolicy.egress`. - Add `extraManifests` and unit coverage for additional rendered manifests. - Make dual-stack, external-db, and external-secrets CI scenarios self-contained for local k3d. - Add network policy unit coverage for baseline, extra egress, and legacy egress compatibility. ## Related - Site sync: helmforgedev/site#353. - Closes #633. ## Validation - `helm template test charts/hoppscotch | rg -n "helm.sh/hook|test-connection|connection-test"` (hook rendered from `templates/tests/connection-test.yaml`) - `helm unittest charts/hoppscotch` (81 tests, 14 suites) - `make template-standards-check CHART=hoppscotch` - `node scripts/charts/validate-chart.js --chart hoppscotch --no-k3d` - `make validate-chart CHART=hoppscotch TIMEOUT=900` (FULLY VALIDATED, 18 layers) - `make site-sync-check CHART=hoppscotch` - `make release-check REPO=charts` - `make attribution-check REPO=charts` ## Site Validation - `npm run lint` - `npm run format:check` - `npm run build`  ## Summary by CodeRabbit * **New Features** * Added `extraManifests` to render additional Kubernetes resources during the Helm release (with templating support). * Added optional `networkPolicy.extraEgress` to append extra egress rules to generated NetworkPolicies. * **Bug Fixes** * Updated CI chart setup to use an in-cluster PostgreSQL service and updated external-secrets sourcing for CI runs. * Adjusted dual-stack service behavior to prefer dual-stack addressing. * **Tests** * Added/updated Helm tests covering extra manifests rendering and NetworkPolicy egress ordering/appending. * **Documentation** * Updated chart docs and `values.schema.json` for `extraManifests` and `extraEgress`.  --------- Co-authored-by: MergeCheck <mergecheck@example.invalid>

mberlofa mentioned this pull request Jul 3, 2026

fix(hoppscotch): align template standards helmforgedev/charts#675

Merged

mberlofa mentioned this pull request Jul 3, 2026

fix: align 81 charts with template standards helmforgedev/charts#633

Closed

mberlofa added 2 commits July 6, 2026 11:46

docs(hoppscotch): sync template standards updates

1898bfb

docs(hoppscotch): document templated extra manifests

e8ad4d3

mberlofa force-pushed the docs/hoppscotch-template-standards-sync branch from 99dcbdb to e8ad4d3 Compare July 6, 2026 14:47

mberlofa merged commit 04d8e33 into main Jul 6, 2026
6 checks passed

mberlofa deleted the docs/hoppscotch-template-standards-sync branch July 6, 2026 14:55

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Uh oh!

docs(hoppscotch): sync template standards updates#353

docs(hoppscotch): sync template standards updates#353
mberlofa merged 2 commits into
mainfrom
docs/hoppscotch-template-standards-sync

mberlofa commented Jul 3, 2026 •

edited by coderabbitai Bot

Loading

Uh oh!

coderabbitai Bot commented Jul 3, 2026 •

edited

Loading

Walkthrough

Changes

Uh oh!

mberlofa commented Jul 5, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Uh oh!

Uh oh!

Conversation

mberlofa commented Jul 3, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Validation

Summary by CodeRabbit

Uh oh!

coderabbitai Bot commented Jul 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Uh oh!

mberlofa commented Jul 5, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

mberlofa commented Jul 3, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jul 3, 2026 •

edited

Loading